Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LOBSTER-Codebeamer tool to use configuration files #137

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

SurajBDeore
Copy link
Contributor

  • Replaced command-line arguments with a YAML-based configuration file for improved usability and maintainability.
  • Simplifies parameter management and enhances readability.

@SurajBDeore SurajBDeore requested a review from a team as a code owner December 3, 2024 11:26
@SurajBDeore SurajBDeore force-pushed the feature/lobster-codebeamer-cmd-to-config branch 5 times, most recently from c8021cc to 824f7e5 Compare December 5, 2024 09:22
- Replaced command-line arguments with a YAML-based configuration file for improved usability and maintainability.
- Simplifies parameter management and enhances readability.
@SurajBDeore SurajBDeore force-pushed the feature/lobster-codebeamer-cmd-to-config branch from 824f7e5 to 4d1e10c Compare December 5, 2024 09:45
@phiwuu phiwuu added the lobster-codebeamer Affects codebeamer integration label Dec 6, 2024
Comment on lines -474 to +473
ap.error("please set CB_ROOT or use --cb-root")
sys.exit("lobster-codebeamer: Please set 'root' in the config file")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I observed that Argument parser error was replaced by sys.exit. Any specific reason for that?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because we are switching from command-line arguments to a config YAML file, it is essentially an argument parser error.

Copy link
Member

@phiwuu phiwuu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a breaking change. Let's build a new release first, and merge afterwards.

using the --schema argument, or configure it in the config file.
2. Configure the 'refs' upstream reference (optional) using --config or
specify directly via command line.
1. Specify the schema of the trace to be generated (optional, defaults to "Requirement"). You can set it using the
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This point is difficult to understand

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lobster-codebeamer Affects codebeamer integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants